Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP - Adiciona checagem de HTML mal formado #3394

Open
wants to merge 1 commit into
base: 3.1.x
Choose a base branch
from

Conversation

edwardoliveira
Copy link
Contributor

Faz uma checagem de HTML mal formado durante o save() de Dispositivo em Textos Articulados.

Descrição

Algumas vezes, é possível que o usuário insira HTMLs mal formados seja por ação ou por vias de copia-e-cola. A intenção deste PR é mitigar as chances disso ocorrer.

Motivação e Contexto

Diminuir a possibilidade de erro em textos articulados por conta de HTML mal formado.

Tipos de Mudanças

  • Bug fix (alteração que corrige uma issue e não altera funcionalidades já existentes)
  • Nova feature (alteração que adiciona uma funcionalidade e não altera funcionalidades já existentes)
  • Alteração disruptiva (Breaking change) (Correção ou funcionalidade que causa alteração nas funcionalidades existentes)

Checklist:

  • Eu li o documento de Contribuição (CONTRIBUTING).
  • Meu código segue o estilo de código deste projeto.
  • Minha alteração requer uma alteração na documentação.
  • Eu atualizei a documentação de acordo.
  • Eu adicionei testes para cobrir minhas mudanças.
  • Todos os testes novos e existentes passaram.

@edwardoliveira edwardoliveira changed the title Adiciona checagem de HTML mal formado WIP - Adiciona checagem de HTML mal formado May 4, 2021
@edwardoliveira
Copy link
Contributor Author

Oi @LeandroJatai, eu tive que resolver um bug de um usuário via OSTicket e daí me veio essa ideia de colocar uma checagem de HTML válido na hora de salvar os Dispositivos. Como vc entende MUITO MAIS desse módulo, visto que pai da criança rsrsrs, poderia me ajudar a validar/corrigir esse PR? 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant